Skip to content

Update to 8.1 stable release#1227

Merged
tianon merged 4 commits into
docker-library:masterfrom
mallardduck:update-8.1rc-to-8.1-stable
Nov 29, 2021
Merged

Update to 8.1 stable release#1227
tianon merged 4 commits into
docker-library:masterfrom
mallardduck:update-8.1rc-to-8.1-stable

Conversation

@mallardduck

Copy link
Copy Markdown
Contributor

Just noticed after doing this that there seems to be a bot that might be making this happen. So please disregard if it's just a matter of time for that to happen to get the 8.1 images updated to release.

@enumag

enumag commented Nov 26, 2021

Copy link
Copy Markdown

It seems that in case of PHP 8.0 the RC -> stable transition was done manually here. So the bot likely doesn't support that case. Your PR should help. :-)

@enumag

enumag commented Nov 26, 2021

Copy link
Copy Markdown

This might need to be adjusted, though I have no clue what this file is about.
https://github.com/docker-library/php/blob/master/generate-stackbrew-library.sh#L4-L7

@mallardduck

Copy link
Copy Markdown
Contributor Author

good catch @enumag! that script seems to be for generating this file: https://github.com/docker-library/official-images/blob/master/library/php

I think I've updated it correctly

@sbuerk

sbuerk commented Nov 27, 2021

Copy link
Copy Markdown

As 8.1 release missing until now - I'm not into it if this PR would really solve it, or if it is the full/right way. Anyway would be nice to have 8.1.0 released as images ready, as we depend on it for our testing.

Not sure who is maintainer on this repository, but looking through the commits maybe @yosifkit ?

@WyriHaximus

Copy link
Copy Markdown

@sbuerk It's the weekend, let the maintainers also enjoy the weekend how they prefer :). I get that you're excited and want an 8.1 stable image available as soon as possible. But generally, these images lag up to a week behind. Also IMHO it might make sense to have #1226 merged in first so we don't end up with a single outdated Alpine 3.13 image of 8.1.

@sbuerk

sbuerk commented Nov 27, 2021

Copy link
Copy Markdown

@WyriHaximus Eh sry .. you are right. (Really it's weekend ?!) I should check my time feeling.
I'm sorry ...

@WyriHaximus

Copy link
Copy Markdown

@sbuerk Hey I get it, we're all tired, we all had our ability to sense time passing distorted, we're all excited about all the features 8.1 brings. The only thing I'm asking is to let maintainers take their rest when they want to without getting pinged, and I'm just assuming the weekend here 😅, so they can go in (well) rested and review PR's like these.

@sbuerk

sbuerk commented Nov 27, 2021

Copy link
Copy Markdown

@WyriHaximus You are absolutly right, and reading my own post myself I'm feeling uncomfortable with it - and I'm regret it.

It was too pressing, so please take my apology.

Very gratefull for this repo and the pre-work. Just slipped in in a 5 min slot and "ping there" before going on with own work and so forth .. which is no excuse !!!

We are "green" wit RC6 , and in generelly it should not change much. Was quick shot. Hopefully you and the maintainer(s) can take the excuse.

@tianon

tianon commented Nov 29, 2021

Copy link
Copy Markdown
Member

Also IMHO it might make sense to have #1226 merged in first so we don't end up with a single outdated Alpine 3.13 image of 8.1.

Agreed -- I've merged #1226 now (and plan to rebase this after I get through more long Thanksgiving weekend email backlog, if @mallardduck doesn't beat me to it 😇 ❤️).

@mallardduck

Copy link
Copy Markdown
Contributor Author

@tianon - i'll get the rebase done RN

@mallardduck
mallardduck force-pushed the update-8.1rc-to-8.1-stable branch from 780aec5 to d233fa9 Compare November 29, 2021 19:10
@tianon
tianon merged commit 8447ef3 into docker-library:master Nov 29, 2021
@tianon

tianon commented Nov 29, 2021

Copy link
Copy Markdown
Member

Nice, thanks! 👍

docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Nov 29, 2021
Changes:

- docker-library/php@8447ef3: Update to 8.1 stable release (docker-library/php#1227)
- docker-library/php@ba79418: Merge pull request docker-library/php#1226 from J0WI/alpine-3.15
- docker-library/php@0c60516: Alpine 3.15
@martinssipenko

martinssipenko commented Nov 30, 2021

Copy link
Copy Markdown

Am I missing something or what but looks like the images with Alpine 3.15 are missing linux/386 and linux/amd64 OS/ARCH type images.

@martinssipenko

Copy link
Copy Markdown

Never mind, I have jumped the gun, looks like the images are still building at the time of writing this comment.

https://doi-janky.infosiftr.net/job/multiarch/job/amd64/job/php/

@enumag

enumag commented Nov 30, 2021

Copy link
Copy Markdown

Is there some way to change the pipelines to build the amd64 and 386 images first? 🤔 It makes no sense that it's starting from the least needed images.

@pherum

pherum commented Nov 30, 2021

Copy link
Copy Markdown
 => [internal] load build definition from Dockerfile                                                                                                           0.0s
 => => transferring dockerfile: 908B                                                                                                                           0.0s
 => [internal] load .dockerignore                                                                                                                              0.0s
 => => transferring context: 2B                                                                                                                                0.0s
 => ERROR [internal] load metadata for docker.io/library/php:8.1-cli-alpine                                                                                    2.3s
 => [auth] library/php:pull token for registry-1.docker.io                                                                                                     0.0s
------
 > [internal] load metadata for docker.io/library/php:8.1-cli-alpine:
------
failed to solve with frontend dockerfile.v0: failed to create LLB definition: no match for platform in manifest sha256:bdf54663bba260b7f61b79dd605b9f17e8a5c3b3765d273a9fcb637cb8930050: not found

@martinssipenko

Copy link
Copy Markdown

@pherum please see comments #1227 (comment) and #1227 (comment).

@WyriHaximus

Copy link
Copy Markdown

Is there some way to change the pipelines to build the amd64 and 386 images first? thinking It makes no sense that it's starting from the least needed images.

@enumag This only happens for the first image of a new minor/major. Honestly, I really doubt changing all that is worth the ~8 hours you get to use the new major/minor earlier once per year.

@tianon

tianon commented Nov 30, 2021 via email

Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants